♻️ refactoring searchable-params test to use test.each #2
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In this pull request, the unit tests for the
SearchParams
class have been significantly refactored. The main change is the use of Jest'stest.each
function, which allows us to run the same test multiple times with different inputs. This makes the tests more concise and easier to maintain, as we can add, remove, or modify test cases simply by modifying an array of parameters.Additionally, the test labels (the descriptions that appear when the tests are run) have been parameterized. This means that the specific values being tested are included in the test label, making it easier to understand what each test is doing and identify which test has failed if there are any failures.
Here's an example of what the new tests look like:
In this example,
%p
in the test label is replaced with the value of entry for each test case. This makes it clear what value is being tested in each run of the test.